Skip to content

CI msys2/ucrt64#749

Open
3rav wants to merge 5 commits intoICLDisco:masterfrom
3rav:master
Open

CI msys2/ucrt64#749
3rav wants to merge 5 commits intoICLDisco:masterfrom
3rav:master

Conversation

@3rav
Copy link

@3rav 3rav commented Apr 3, 2025

No description provided.

@3rav 3rav mentioned this pull request Apr 3, 2025
Open
@abouteiller
Copy link
Contributor

abouteiller commented May 8, 2025

@G-Ragghianti we don't have any windows runners do we?

@G-Ragghianti
Copy link
Contributor

G-Ragghianti commented May 8, 2025

We do not (thank goodness).

@3rav
Copy link
Author

3rav commented May 12, 2025

After last update:

[94/468] Building C object parsec/CMakeFiles/parsec.dir/parsec.c.obj
FAILED: parsec/CMakeFiles/parsec.dir/parsec.c.obj 
D:\a\_temp\msys64\ucrt64\bin\cc.exe -DBUILDING_PARSEC -DNDEBUG -DYYERROR_VERBOSE -Dparsec_EXPORTS -ID:/a/parsec/parsec/build/parsec/include -ID:/a/parsec/parsec/build -ID:/a/parsec/parsec/parsec/include -ID:/a/parsec/parsec -ID:/a/parsec/parsec/build/parsec/data_dist/matrix -ID:/a/parsec/parsec/build/parsec/data_dist/matrix/redistribute -D_GNU_SOURCE -Wno-attributes -Wno-incompatible-pointer-types -Wno-implicit-function-declaration -O3 -DNDEBUG -std=gnu11 -Wall -Wextra -fdiagnostics-color -MD -MT parsec/CMakeFiles/parsec.dir/parsec.c.obj -MF parsec\CMakeFiles\parsec.dir\parsec.c.obj.d -o parsec/CMakeFiles/parsec.dir/parsec.c.obj -c D:/a/parsec/parsec/parsec/parsec.c
In file included from D:/a/parsec/parsec/parsec/parsec.c:35:
D:/a/parsec/parsec/parsec/remote_dep.h: In function 'remote_dep_rank_to_bit':
D:/a/parsec/parsec/parsec/remote_dep.h:429:25: error: 'parsec_remote_dep_context' undeclared (first use in this function); did you mean 'parsec_remote_dep_on'?
  429 |     uint32_t nb_nodes = parsec_remote_dep_context.max_nodes_number;
      |                         ^~~~~~~~~~~~~~~~~~~~~~~~~
      |                         parsec_remote_dep_on
D:/a/parsec/parsec/parsec/remote_dep.h:429:25: note: each undeclared identifier is reported only once for each function it appears in
D:/a/parsec/parsec/parsec/remote_dep.h: In function 'remote_dep_bit_to_rank':
D:/a/parsec/parsec/parsec/remote_dep.h:438:20: error: 'parsec_remote_dep_context' undeclared (first use in this function); did you mean 'parsec_remote_dep_on'?
  438 |     int nb_nodes = parsec_remote_dep_context.max_nodes_number;
      |                    ^~~~~~~~~~~~~~~~~~~~~~~~~
      |                    parsec_remote_dep_on
D:/a/parsec/parsec/parsec/parsec.c: In function '__parsec_thread_init':
D:/a/parsec/parsec/parsec/parsec.c:346:17: warning: cast to pointer from integer of different size [-Wint-to-pointer-cast]
  346 |     void *ret = (void*)(long)__parsec_context_wait(es);
      |                 ^
D:/a/parsec/parsec/parsec/parsec.c: In function 'parsec_taskpool_sync_ids_context':
D:/a/parsec/parsec/parsec/parsec.c:2113:49: warning: unused parameter 'comm' [-Wunused-parameter]
 2113 | void parsec_taskpool_sync_ids_context( intptr_t comm )
      |                                        ~~~~~~~~~^~~~
D:/a/parsec/parsec/parsec/parsec.c: In function 'parsec_taskpool_sync_ids':
D:/a/parsec/parsec/parsec/parsec.c:2144:47: error: 'MPI_COMM_WORLD' undeclared (first use in this function)
 2144 |   parsec_taskpool_sync_ids_context( (intptr_t)MPI_COMM_WORLD );
      |                                               ^~~~~~~~~~~~~~
[95/468] Building C object parsec/CMakeFiles/parsec.dir/parsec_reshape.c.obj
FAILED: parsec/CMakeFiles/parsec.dir/parsec_reshape.c.obj 
D:\a\_temp\msys64\ucrt64\bin\cc.exe -DBUILDING_PARSEC -DNDEBUG -DYYERROR_VERBOSE -Dparsec_EXPORTS -ID:/a/parsec/parsec/build/parsec/include -ID:/a/parsec/parsec/build -ID:/a/parsec/parsec/parsec/include -ID:/a/parsec/parsec -ID:/a/parsec/parsec/build/parsec/data_dist/matrix -ID:/a/parsec/parsec/build/parsec/data_dist/matrix/redistribute -D_GNU_SOURCE -Wno-attributes -Wno-incompatible-pointer-types -Wno-implicit-function-declaration -O3 -DNDEBUG -std=gnu11 -Wall -Wextra -fdiagnostics-color -MD -MT parsec/CMakeFiles/parsec.dir/parsec_reshape.c.obj -MF parsec\CMakeFiles\parsec.dir\parsec_reshape.c.obj.d -o parsec/CMakeFiles/parsec.dir/parsec_reshape.c.obj -c D:/a/parsec/parsec/parsec/parsec_reshape.c
In file included from D:/a/parsec/parsec/parsec/parsec_reshape.c:14:
D:/a/parsec/parsec/parsec/remote_dep.h: In function 'remote_dep_rank_to_bit':
D:/a/parsec/parsec/parsec/remote_dep.h:[429](https://github.com/3rav/parsec/actions/runs/14965782012/job/42035417773#step:4:430):25: error: 'parsec_remote_dep_context' undeclared (first use in this function); did you mean 'parsec_remote_dep_on'?
  429 |     uint32_t nb_nodes = parsec_remote_dep_context.max_nodes_number;
      |                         ^~~~~~~~~~~~~~~~~~~~~~~~~
      |                         parsec_remote_dep_on
D:/a/parsec/parsec/parsec/remote_dep.h:429:25: note: each undeclared identifier is reported only once for each function it appears in
D:/a/parsec/parsec/parsec/remote_dep.h: In function 'remote_dep_bit_to_rank':
D:/a/parsec/parsec/parsec/remote_dep.h:438:20: error: 'parsec_remote_dep_context' undeclared (first use in this function); did you mean 'parsec_remote_dep_on'?
  [438](https://github.com/3rav/parsec/actions/runs/14965782012/job/42035417773#step:4:439) |     int nb_nodes = parsec_remote_dep_context.max_nodes_number;
      |                    ^~~~~~~~~~~~~~~~~~~~~~~~~
      |                    parsec_remote_dep_on
D:/a/parsec/parsec/parsec/parsec_reshape.c: In function 'parsec_set_up_reshape_promise':
D:/a/parsec/parsec/parsec/parsec_reshape.c:454:85: error: 'MPI_COMM_WORLD' undeclared (first use in this function)
  454 |             MPI_Pack_size(aux_data.remote.dst_count, aux_data.remote.dst_datatype , MPI_COMM_WORLD, &dsize);
      |                                                                                     ^~~~~~~~~~~~~~
[96/468] Building C object parsec/CMakeFiles/parsec.dir/data.c.obj
FAILED: parsec/CMakeFiles/parsec.dir/data.c.obj 
D:\a\_temp\msys64\ucrt64\bin\cc.exe -DBUILDING_PARSEC -DNDEBUG -DYYERROR_VERBOSE -Dparsec_EXPORTS -ID:/a/parsec/parsec/build/parsec/include -ID:/a/parsec/parsec/build -ID:/a/parsec/parsec/parsec/include -ID:/a/parsec/parsec -ID:/a/parsec/parsec/build/parsec/data_dist/matrix -ID:/a/parsec/parsec/build/parsec/data_dist/matrix/redistribute -D_GNU_SOURCE -Wno-attributes -Wno-incompatible-pointer-types -Wno-implicit-function-declaration -O3 -DNDEBUG -std=gnu11 -Wall -Wextra -fdiagnostics-color -MD -MT parsec/CMakeFiles/parsec.dir/data.c.obj -MF parsec\CMakeFiles\parsec.dir\data.c.obj.d -o parsec/CMakeFiles/parsec.dir/data.c.obj -c D:/a/parsec/parsec/parsec/data.c
In file included from D:/a/parsec/parsec/parsec/data.c:16:
D:/a/parsec/parsec/parsec/remote_dep.h: In function 'remote_dep_rank_to_bit':
D:/a/parsec/parsec/parsec/remote_dep.h:429:25: error: 'parsec_remote_dep_context' undeclared (first use in this function); did you mean 'parsec_remote_dep_on'?
  429 |     uint32_t nb_nodes = parsec_remote_dep_context.max_nodes_number;
      |                         ^~~~~~~~~~~~~~~~~~~~~~~~~
      |                         parsec_remote_dep_on
D:/a/parsec/parsec/parsec/remote_dep.h:429:25: note: each undeclared identifier is reported only once for each function it appears in
D:/a/parsec/parsec/parsec/remote_dep.h: In function 'remote_dep_bit_to_rank':
D:/a/parsec/parsec/parsec/remote_dep.h:438:20: error: 'parsec_remote_dep_context' undeclared (first use in this function); did you mean 'parsec_remote_dep_on'?
  438 |     int nb_nodes = parsec_remote_dep_context.max_nodes_number;
      |                    ^~~~~~~~~~~~~~~~~~~~~~~~~
      |                    parsec_remote_dep_on
[97/468] Building C object parsec/CMakeFiles/parsec.dir/data_distribution.c.obj
ninja: build stopped: subcommand failed.

@3rav 3rav marked this pull request as ready for review May 12, 2025 07:44
@3rav 3rav requested a review from bosilca as a code owner May 12, 2025 07:44
@G-Ragghianti
Copy link
Contributor

G-Ragghianti commented May 12, 2025

It looks like the github-hosted runners offer this windows instance (windows-latest). If we approve the PR, we can see if it will run on github's infrastructure. I don't know if there is a fee or what the limits are.

bosilca
bosilca reviewed May 12, 2025
View reviewed changes
@bosilca
Update .github/workflows/msys2.yml
8b24af0 
Move this on our github runner
@bosilca
Copy link
Contributor

bosilca commented May 12, 2025

We have some runners available on github. I moved this on our github runner, let's see how it goes. I means obviously it will fail as we can't build without MPI, but at least we will see if this works.

@bosilca
Copy link
Contributor

bosilca commented May 13, 2025

No msys2-ucrt64 runner from github picked this job up and there is no info on the timeout. Not sure how to move this forward.

@G-Ragghianti
Copy link
Contributor

G-Ragghianti commented May 13, 2025 •
edited
Loading

There is an error on the organization level of the runner config:

Runners are shut down due to spending limit caps being reached. Please try raising your Actions spending limit to continue using these runners. See billing page for more details. If you’re still experiencing problems, please reach out to our support.

@bosilca
Copy link
Contributor

bosilca commented May 13, 2025

I don't see any github runner having done anything here, I don't grok what caps are we reaching. The repo is public, I use a public runner and here it is clearly mentionned that:

The use of these runners on public repositories is free and unlimited.

bosilca
bosilca reviewed May 13, 2025
View reviewed changes
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@bosilca bosilca bosilca approved these changes

At least 2 approving reviews are required to merge this pull request.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@3rav @abouteiller @G-Ragghianti @bosilca